-
-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Show blog post cover image on the main site #424
Conversation
@moteanup24 is attempting to deploy a commit to the OpenSats Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Awesome, thank you 🙏 Looks great already! Can we try to see how it would look like if we made the image a little bigger and move the date "inside" the image, and make the text of the date white? |
Nice! I like that. Can we try moving the date down a little bit (so that it aligns with the title) and to the right a little (so that it aligns with the logo)? Would love to hear some other opinions in terms of how it looks @BoltTouring @Arvin21M |
Re: Date On Image
Re: Incorporating The Header Image
TLDR:
🙏 |
Thanks @Arvin21M! The comment thread might be a little hard to follow so to be sure, here is the status of the current PR: |
Nice! I think that looks really good! One last thing: clicking on the image should open up the blog post. Can you please add that? |
fd18842
to
f0c5066
Compare
Apologies was tied up with family stuff for a few days, but pushed this change last night! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done! Looks good to me ✅
Open for comments/requests!
Build preview: